Skip to content

Comments

Move regular expressions#13

Open
Rosey wants to merge 23 commits intosstur:masterfrom
Rosey:move_regular_expressions
Open

Move regular expressions#13
Rosey wants to merge 23 commits intosstur:masterfrom
Rosey:move_regular_expressions

Conversation

@Rosey
Copy link

@Rosey Rosey commented Jul 16, 2016

Hello! Hopefully this is helpful. I noticed the regular expressions weren't being saved in a variable anywhere and were being created inside a map loop so decided to change that. (general stack overflow comment that talks about this http://stackoverflow.com/a/32523333) and then made a new file to contain them and wrote some tests while I was at it...

Freddy03h and others added 23 commits March 24, 2016 15:59
Support headers 4, 5 and 6
Requires Image enum value from `draft-js-utils`.
Support image export based on entity attributes
Fixed support for image type entity
per package.json and the contents of the LICENSE file
In the interest of improving regular expression performance, it makes sense to
create each regular expression once, instead of re-creating every loop of `map`.

These utility regular expressions and the methods that use them seemed better
suited to their own file with documentation, so they can be tested individually.
@sstur sstur force-pushed the master branch 2 times, most recently from 255f0a8 to f6c1e51 Compare July 27, 2017 01:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants